Skip to content

feat(devmanual): Document contacts menu basics#11222

Merged
ChristophWurst merged 1 commit into
masterfrom
feat/devmanual/contacts-menu
Oct 18, 2023
Merged

feat(devmanual): Document contacts menu basics#11222
ChristophWurst merged 1 commit into
masterfrom
feat/devmanual/contacts-menu

Conversation

@ChristophWurst

@ChristophWurst ChristophWurst commented Oct 17, 2023

Copy link
Copy Markdown
Member

☑️ Resolves

🖼️ Screenshots

Screenshot 2023-10-18 at 10-49-56 Contacts Menu — Nextcloud latest Developer Manual latest documentation

@christianlupus christianlupus left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general, I think, this looks good. I added some comments, that might be addressed ideally.

Comment thread developer_manual/digging_deeper/groupware/contacts_menu.rst Outdated
Contacts Menu
=============

Nextcloud shows a *Contacts menu* in the right corner of the header. This menu lists a user's contacts. These contact entries can be extended by apps.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An image (as an example might be nice here).

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a fan of too many screenshots in the docs because they are getting outdated so fast and redoing all screenshots for a release is a lot of work

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was just a suggestion. Feel free to resolve this comment.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst force-pushed the feat/devmanual/contacts-menu branch from 3519fe1 to c7c3352 Compare October 18, 2023 10:27
@ChristophWurst ChristophWurst merged commit 8e13ded into master Oct 18, 2023
@ChristophWurst ChristophWurst deleted the feat/devmanual/contacts-menu branch October 18, 2023 10:42
@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable27

@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable26

@ChristophWurst

Copy link
Copy Markdown
Member Author

/backport to stable27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants